Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

[ci] only run in PaddlePaddle/PaddleSOT repo #221

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

gouzil
Copy link
Member

@gouzil gouzil commented Jun 29, 2023

让ci不会在fork的仓库上运行

@paddle-bot
Copy link

paddle-bot bot commented Jun 29, 2023

Thanks for your contribution!

@paddle-bot paddle-bot bot added contributor External developers status: proposed labels Jun 29, 2023
@gouzil gouzil marked this pull request as draft June 29, 2023 08:20
@SigureMo
Copy link
Member

SigureMo commented Jun 29, 2023

actions/checkout 默认就有这功能吧?

比如 #220 https://github.com/PaddlePaddle/PaddleSOT/actions/runs/5409796119/jobs/9830376385?pr=220

image
  • 其中 9fff894 是 PR 的 hash
  • 其中 94fd654 是 develop 的 hash
  • 其中 30473f1 是 merge PR into develop 的 hash

@gouzil gouzil changed the title [ci] add merge check [ci] only run in PaddlePaddle/PaddleSOT repo Jun 29, 2023
@gouzil
Copy link
Member Author

gouzil commented Jun 29, 2023

actions/checkout 默认就有这功能吧?

比如 #220 https://github.com/PaddlePaddle/PaddleSOT/actions/runs/5409796119/jobs/9830376385?pr=220

image * 其中 [9fff894](https://github.com/PaddlePaddle/PaddleSOT/commit/9fff894f4d43b1f2f04dbe35d097cd897f2ca80b) 是 PR 的 hash * 其中 [94fd654](https://github.com/PaddlePaddle/PaddleSOT/commit/94fd654d92b51e0a84500caf15fa4af9b5410a8f) 是 develop 的 hash * 其中 [30473f1](https://github.com/PaddlePaddle/PaddleSOT/commit/30473f105f37bc218e874bcedc8417b9f3248c5e) 是 merge PR into develop 的 hash

是的是的, 我换了一个修

@gouzil gouzil marked this pull request as ready for review June 29, 2023 12:26
@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

这个看起来倒是没什么问题,只是这个是有什么需求吗?一般只有给 Fork 的 repo 提 PR 才会触发 CI?一旦我们以后需要在 fork 的 repo 里触发 CI 呢?

@gouzil
Copy link
Member Author

gouzil commented Jul 6, 2023

这个看起来倒是没什么问题,只是这个是有什么需求吗?一般只有给 Fork 的 repo 提 PR 才会触发 CI?一旦我们以后需要在 fork 的 repo 里触发 CI 呢?

可以看一下其他人的仓库, 比如 https://github.com/zrr1999/paddle-symbolic-trace, 可以发现merge后会再运行一次ci (不过我之前以为是会占用限时,现在看来好像不会)

@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

咦?我刚看你的 repo 是没有的啊

@SigureMo
Copy link
Member

SigureMo commented Jul 6, 2023

啊,还真有

@gouzil
Copy link
Member Author

gouzil commented Jul 6, 2023

咦?我刚看你的 repo 是没有的啊

因为我手动把actions关了

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants